Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Rename binary and default directory #193

Merged
merged 4 commits into from
Feb 15, 2024

Conversation

hacheigriega
Copy link
Member

@hacheigriega hacheigriega commented Feb 6, 2024

Explanation of Changes

  • Binary name has been renamed from seda-chaind to sedad
  • Default directory name has been renamed from .seda-chain to .sedad

@hacheigriega hacheigriega changed the title refactor: binary and directory names change refactor: Rename binary and default directory Feb 6, 2024
@hacheigriega hacheigriega requested a review from a team February 6, 2024 13:49
@hacheigriega hacheigriega force-pushed the hy/daemon-name-change branch 2 times, most recently from 63f605a to 4126844 Compare February 6, 2024 15:55
Copy link
Contributor

@gluax gluax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but one concern I have is if we are renaming the home directory the chain uses to ~/.seda and if a person runs both a chain and an overlay node on the same PC they are now using the same directory. In this case, I would be concerned about logs overwriting each other from either binary and user UX, but I don't know if a person is likely to run both on the same machine. I don't see why not, considering the overlay is very light. Maybe we should consider ~/.seda/chain on the chain side and ~/.seda/overlay on the overlay side.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to commit this empty file?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. It's common for node operators to do rm -rf ~/.seda which would remove the overlay data dir with it.

There is actually a simpler approach, simply do ~/.sedad like osmosis does (~/.osmosisd).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it! 👍

@mariocao
Copy link
Member

mariocao commented Feb 6, 2024

Approving, but one concern I have is if we are renaming the home directory the chain uses to ~/.seda and if a person runs both a chain and an overlay node on the same PC they are now using the same directory. In this case, I would be concerned about logs overwriting each other from either binary and user UX, but I don't know if a person is likely to run both on the same machine. I don't see why not, considering the overlay is very light. Maybe we should consider ~/.seda/chain on the chain side and ~/.seda/overlay on the overlay side.

I don't mind changing the folder as long as we make sure that they is no overwriting. 😅

@Thomasvdam
Copy link
Member

We should also go over our other projects/docs and update those. The contracts repo has some references that I know of and there's probably more.

@hacheigriega hacheigriega merged commit 5fa0b6f into main Feb 15, 2024
14 of 16 checks passed
@hacheigriega hacheigriega deleted the hy/daemon-name-change branch February 15, 2024 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants